Merge branch 'gtk-3-22' into 'gtk-3-22'

gtkprintbackendcups.c: fix \n at end of a debugging note

See merge request GNOME/gtk!24

Merge branch 'gtk-debug-flags-with-open-display' into 'gtk-3-22'

gtk_init: Fix debug flags handling when a display is already open

See merge request GNOME/gtk!26

Updated Lithuanian translation

Update German translation

Merge branch 'gtk-3-22' into 'gtk-3-22'

Fix annotation for gtk_tree_view_is_blank_at_pos()

See merge request GNOME/gtk!12

gtk_init: Fix debug flags handling when a display is already open

In PyGObject gdk_init() is called before gtk_init() and thus there is
already a default display open when gtk_init() is called.
The code assigning the display to the debug_flags struct gets only
called when the default display changes, which never happens
when there already is one. As a result GTK_DEBUG=interactive
doesn't do anyting with Python apps.

This makes it call the change callback in case a display is already
there.

See https://gitlab.gnome.org/GNOME/pygobject/issues/166

Fix incorrect optimization in find_builtin_icon().

When we found an icon with exactly the requested size, we'd stop
searching immediately (good), but we'd neglect to set the returned
min_difference to 0 (bad).  This caused theme_lookup_icon() to
prefer other, potentially much worse, matches over the exact one.

docs: Add proper deprecation text for GtkStatusIcon

We link to the HowDoI for GNotification in the class description, but we
should be more verbose in the deprecation notices for each function of
the GtkStatusIcon class.

https://bugzilla.gnome.org/show_bug.cgi?id=743975

gtkprintbackendcups.c: fix \n at end of a debugging note

(cherry picked from commit 7461ceebe34bfc6f882145e89e1ba2f187d08303)

Update Polish translation

Update Croatian translation

Update Croatian translation

Update Brazilian Portuguese translation

Update Polish translation

Update Swedish translation

gtkplacessidebar: Rename favorites to starred

To have more consistency in the name. The same was done in Nautilus in
commit https://gitlab.gnome.org/GNOME/nautilus/commit/27b039b37c1149fc8abbef6a683dd939e6f2bf1f.

Update POTFILES.in

Merge branch 'wip/carlosg/imwayland-3-22' into 'gtk-3-22'

modules: Add wayland IM implementation

See merge request GNOME/gtk!5

3.22.28

modules: Add wayland IM implementation

This IM context implementation goes through the gtk-text-input protocol,
leaving up to the compositor the actual interaction with IM engines. If
the protocol is not offered by the compositor, GTK+ will fallback to the
IMs as specified through GtkSettings.

gdk/wayland: Add GdkDisplay call to query available globals

The internal known_globals hashtable is used to carry accounting for
interfaces that depend on others (as ordering is not guaranteed), extend
its usage so it also keeps track of unimplemented interfaces (here at
least).

The API call will then use this to allow querying the globals offered by
the compositor, it will be useful to determine whether we can use
text-input protocols or should fallback to other IMs.

Fix irregular gdk_frame_clock_get_frame_time

This fixes stuttering in animations that rely on the regularity of
gdk_frame_clock_get_frame_time.

https://bugzilla.gnome.org/show_bug.cgi?id=787665

BEFORE
gdkgears:
58 FPS and visibly stuttering
gnome-maps on a 59.95Hz monitor:
"paint" g_get_monotonic_time +17278μs, gdk_frame_clock_get_frame_time +17278μs
"paint" g_get_monotonic_time +17449μs, gdk_frame_clock_get_frame_time +17426μs
"paint" g_get_monotonic_time +17620μs, gdk_frame_clock_get_frame_time +17600μs

AFTER
gdkgears:
60 FPS and smoother
gnome-maps on a 59.95Hz monitor:
"paint" g_get_monotonic_time +18228μs, gdk_frame_clock_get_frame_time +16680μs
"paint" g_get_monotonic_time +15010μs, gdk_frame_clock_get_frame_time +16680μs
"paint" g_get_monotonic_time +17134μs, gdk_frame_clock_get_frame_time +16680μs

Fix annotation for gtk_tree_view_is_blank_at_pos()

The annotation (allow-none) is wrong. Since
gtk_tree_view_is_blank_at_pos() also calls
gtk_tree_view_get_path_at_pos(), the same fields should have the same
annotations.

Add missing private header to the build

Closes: #26
        https://bugzilla.gnome.org/show_bug.cgi?id=793442

Update Estonian translation

3.22.27

Update Estonian translation

Merge branch 'gtk-3-22' into 'gtk-3-22'

calendar: Use the new "%OB" format if supported

See merge request GNOME/gtk!2

calendar: Use the new "%OB" format if supported

Due to the recent changes introduced in glibc 2.27 "%OB" is the
correct format to obtain a month name as used in the calendar
header.  The same rule has been working in BSD family (including
OS X) since 1990s.  This simple hack checks whether "%OB" is supported
at runtime and uses it if it is, falls back to the old "%B" otherwise.

Closes: #9

x11: Don't call XInput API for core events

Fixes emacs crashing with XMing.

https://bugzilla.redhat.com/show_bug.cgi?id=1483942

Update Estonian translation

Update Greek translation

docs: Use the appropriate deprecation for properties

The GtkFontButton:font-name property has been deprecated, but it's using
a non-standard annotation in its documentation.

gtk: Cast g_object_ref() to the right type

In GLib 2.56, g_object_ref() will check that you're assigning the return
value to a variable of the same type you're passing in.

Do not use deprecated error trap API

Use the X11-specific API instead.

toolbar: Use gdk_display_flush()

Instead of the deprecated gdk_flush().

selection: Disable deprecation warnings

We are using a lot of deprecated API, and we know it.

Since the selection code is going to be replaced in GTK 4.0, there's no
real point in keeping the warnings enabled in 3.22.

placessidebar: Move declaration inside conditional block

The `sidebar` variable is only used by the conditional block for
libcloudprovider.

Ignore deprecations for gdk_flush()

When terminating the main loop, we're really trying to flush all
GdkDisplay connections, so it's actually a legitimate internal use
case.

docs: Properly deprecate gdk_keymap_get_default()

We're using a compiler annotation, but there's no deprecation notice in
the gtk-doc stanza.

accelgroup: Do not use deprecated API

Use gdk_keymap_get_for_display() with the default display, instead of
the deprecated gdk_keymap_get_default().

deprecated: Cast g_object_ref() to the right type

In GLib 2.56, g_object_ref() will check that you're assigning the return
value to a variable of the same type you're passing in.

a11y: Cast g_object_ref() to the right type

In GLib 2.56, g_object_ref() will check that you're assigning the return
value to a variable of the same type you're passing in.

Remove stray format character

We are not using any string in the debug message.

broadway: Cast g_object_ref() to the right type

In GLib 2.56, g_object_ref() will check that you're assigning the return
value to a variable of the same type you're passing in.

x11: Balance an error trap

We are pushing an error trap, and never popping it from the stack.

Fix overzalous method annotations for drag and drop

Functional revert of commit 9c4892f2915.

Fixes introspection scanner warnings like:

  Warning: Gtk: gtk_drag_finish: Methods must belong to the same
  namespace as the class they belong to

That is, the gtk_drag_* functions cannot be methods as they have a
"GdkDragContext" as the instance parameter, and that is not a valid
type for the Gtk namespace.

This is not an introspected ABI change, as the generated introspection
data ignores the annotation.

Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>
https://bugzilla.gnome.org/show_bug.cgi?id=692152

Update Swedish translation

Update Polish translation

Add AGPL3-only licence to GtkAboutDialog

https://bugzilla.gnome.org/show_bug.cgi?id=792793

Update Friulian translation

Update Friulian translation

Update Galician translation

Update Galician translation

Update Chinese (Taiwan) translation

Update Chinese (Taiwan) translation

Merge branch 'wip/carlosg/fix-issue-1' into 'gtk-3-22'

wayland: Improve EOF detection when reading selections

See merge request GNOME/gtk!1

wayland: Improve EOF detection when reading selections

g_input_stream_read_bytes() roughly provides the same guarantees
than g_input_stream_read() wrt the number of bytes being possibly
read (i.e. it being a best effort, but no real guarantees).

Instead, rely on the 0-len read that we'd get at the end of the
transfer.

Fixes clipboard/DnD transfers possibly being cut short, resulting
on "Broken pipe" errors on the other side.

https://gitlab.gnome.org/GNOME/gtk/issues/1

Closes: #1

cssprovider: Only free bytes when they exist

css: Add a workaround for gtk_widget_override_font()

The problem here is that the CSS machinery expects font sizes to be in
pixels, but gtk_widget_override_font() provides a value in point and the
CSS machinery has no ability to query the DPI and convert.

This patch changes the dconversion DPI we use from a hardcoded 96 to the
default screen's DPI, which should work better than before.
This will of course not listen to changes in the default screen's DPI,
but that shouldn't be a problem.

People who want to workaround this should use gtk_widget_override_font()
with a font that has an absolute size set via
pango_font_description_set_absolute_size (size * PANGO_SCALE *
                                          gdk_screen_get_resolution (screen));

https://bugzilla.gnome.org/show_bug.cgi?id=774248

Update Finnish translation

Update Catalan translation

Update Catalan translation

iconview: Fix a problem with the previous change

After commit ffef28a7e8d5ffef7de6a3baccb30b0021b6b0ff,
gtk-icon-browser was spewing critical warnings when
changing sections. Avoid that by respecting the return
value of gtk_tree_model_get_iter.

wayland: Add support for BTN_STYLUS3

BTN_STYLUS3 is defined by the Linux 4.15 kernel and is sent when the
third button on a stylus is pressed. At the moment, only Wacom's "Pro
Pen 3D" has three stylus buttons. Pressing this button triggers a button
8 event to be sent under X11, so we use the same mapping here.

https://bugzilla.gnome.org/show_bug.cgi?id=790033

tests: Don't add testswitch twice in Makefile.am

menu: Guard against NULL toplevel

This can happen, as indicated by GtkMenu explicitly connecting to
::destroy of its toplevel window. Do the same thing in GtkComboBox.

Update Afrikaans translation

Update Afrikaans translation

Widget: Don’t call reset() on NULL EventController

GtkGesture is a GtkEventController. gtk_event_controller_dispose() calls
_gtk_widget_remove_controller(). That NULLs the pointer-to-Controller in
our EventControllerData but does not delete said ECData from our GList.

Subsequently, if that same Widget gets unparent()ed, that method calls
unset_state_flags(), which leads to doing reset_controllers() if we are
insensitive. Now, unlike most most other loops over the GList of ECData,
reset_controllers() does not skip nodes whose pointer-to-Controller is
NULL. So, we call gtk_event_controller_reset(NULL) and get a CRITICAL.

This surfaced in a gtkmm program. The Gesture is destroyed before the
Widget. The Widget then gets dispose()d, which calls unparent()… boom.
I didn’t find an MCVE yet but would hope this logic is correct anyway:

The simplest fix is to make the loop in gtk_widget_reset_controllers()
skip GList nodes with a NULL Controller pointer, like most other such
loops, so we avoid passing the NULL to gtk_event_controller_reset().

In other, live cases, _gtk_widget_run_controllers() loops over the GList
and removes/frees nodes having NULL Controllers, so that should suffice.
But this clearly was not getting a chance to happen in the failing case.

https://bugzilla.gnome.org/show_bug.cgi?id=792624

Update Esperanto translation

GtkGestureMultiPress: check event state before emiting released signal

Fix bug 771986 "Inconsistent 'row-activated' signal emission before \
drag'n'drop, 'activate-on-single-click'=TRUE, 'reorderable'=TRUE"

Update Hungarian translation

Update Hungarian translation

iconview: Fix updates with filter models

Filter models rely on views taking a ref on every node
they care about. GtkIconView was not doing that. Amazingly,
this has never shown up in a bug so far, until I spotted
the fallout in gnome-font-viewer.

Add a test for ::row-changed vs node refs

Test that filter models propagate ::row-changed if there is
an external reference on the node, and not otherwise. This
is showing up in buggy icon view behaviour, where the icon
view is not redrawing if the content changes in a model that
is below a filter model.

font chooser: Fix property notification

We must notify the font and font-desc properties when the
list selection changes, and return NULL values for them
when there is no selection in the list.

font chooser dialog: Improve sensitivity handling

Enable the select button when a font is selected, as it should be.

font chooser: Small revision of the UI

Show only the font names in the list, in their own
font, in order to make the list less noisy.

gtk/queryimmodules.c: Make the output deterministic.

Whilst working on the Reproducible Builds effort [0], we noticed that
queryimmodules generates non-reproducible output as it iterates over the
filesystem without sorting.

Patch attached.

 [0] https://reproducible-builds.org/

Signed-off-by: Chris Lamb <lamby@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=786528

tree model sort: Fix set_sort_column

We were failing to change the sort order for the
default sort column in some cases. Fix that, and
add a testcase for this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=792459

Add a testcase for the previous fix

tree model sort: Fix initial default sort func

gtk_tree_sortable_has_default_sort_func should return
FALSE initially.

Update Polish translation

emoji: Skip overly wide fallback rendering

Some emoji fonts (such as Emoji One), render Emoji sequences
such as some of the family variations using multiple individual
glyphs. This rendering is too wide and breaks our grid layout.
Therefore, we will just skip any sequence whose rendering is
more than twice as wide as a simple smiley.

GdkMonitor: Fix link to nonexistent Display method

There is no gdk_display_get_monitors(). Instead, we have to use
gdk_display_get_n_monitors() and gdk_display_get_monitor(int).

HeaderBar: Explain use in conjunction w/ GtkWindow

After hinting how good GtkHeaderBar is for GtkWindow, let’s link to
the latter and indicate how users can make the two work together.

Window: Mention GtkHeaderBar in set_titlebar() doc

This is the typical thing passed here and what most users want, so we
should mention it here, rather than requiring users to figure it out.

Window: Clarify resize() doc about titlebar widget

Clarify the reference to HeaderBar, as it applies to any custom title
widget; HeaderBar is only the most common one used. Also, fix a typo.

wayland: Use portable sed syntax in gdk/wayland/Makefile.am

Replace non-portable \<, \>, \+ with \(, \), \{, \} to avoid build
failure on systems not using GNU sed.

https://bugzilla.gnome.org/show_bug.cgi?id=792279

GtkCssImageSurface: add cache for the last drawn size.

Keep a copy of the scaled image to speed up rendering multiple times
the image at the same size.

Adwaita: Fix typo lenght => length in SASS

Update Icelandic translation

Update Icelandic translation

ScrolledWindow: add() before remove() in snippet

We can't remove() a child widget that wasn't already add()ed, of course.

file button: Don't leak rows

The file chooser button manually manages the memory of
data in its model, so it needs to explicitly free the
rows.

Remove a dead macro

GCs have been dead for a long time.

Make a few more code samples compile

entry: Remove some questionable code examples

widget: Make all code snippets compile

liststore: Make a code snippet compile